Skip to content

change: remove all opencloud_full references#454

Merged
micbar merged 2 commits intomainfrom
remove-opencloud_full
Sep 9, 2025
Merged

change: remove all opencloud_full references#454
micbar merged 2 commits intomainfrom
remove-opencloud_full

Conversation

@micbar
Copy link
Member

@micbar micbar commented Sep 5, 2025

Todos

  • Remove all references to opencloud_full
  • Change the guides to use the correct compose files

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes all references to opencloud_full and replaces them with the correct opencloud-compose repository references. The changes update documentation and configuration examples to point to the new repository location and naming convention.

  • Updated repository references from opencloud_full to opencloud-compose
  • Modified Docker volume names and container references to use the new naming
  • Removed outdated example path references in configuration documentation

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
static/env-vars/search_readme.md Updated link reference from opencloud_full to opencloud-compose repository
static/env-vars/proxy_readme.md Updated CSP configuration example path reference
static/env-vars/app-registry_readme.md Removed reference to opencloud_full example path
docs/admin/resources/common-issues.md Updated Docker volume names and container references
docs/admin/maintenance/migrate.md Updated Docker container name reference
docs/admin/configuration/radicale-integration.md Updated deployment example references throughout
docs/admin/configuration/authentication-and-user-management/keycloak.md Updated multiple repository and configuration path references

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@micbar micbar marked this pull request as draft September 5, 2025 14:56
@schweigisito schweigisito marked this pull request as ready for review September 8, 2025 06:11
@schweigisito
Copy link
Member

I haven't found any further reference to opencloud_full.

Copy link
Member

@schweigisito schweigisito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't found any further references to opencloud_full.

@micbar micbar force-pushed the remove-opencloud_full branch from 6e0feb1 to 3974f02 Compare September 8, 2025 20:14
@micbar
Copy link
Member Author

micbar commented Sep 8, 2025

@schweigisito I found more leftovers inside the guides. Now it looks better.

@Svanvith Please check my guide modifications.

@schweigisito
Copy link
Member

@schweigisito I found more leftovers inside the guides. Now it looks better.

@Svanvith Please check my guide modifications.

Good catch. I have only checked for old terms and references, but didn‘t give attention that the installation logic also changes though.

@Svanvith
Copy link
Contributor

Svanvith commented Sep 9, 2025

Looks right to me.
Keycloak is checked ;)

@micbar micbar merged commit 4b6f1cb into main Sep 9, 2025
2 checks passed
@micbar micbar deleted the remove-opencloud_full branch September 9, 2025 14:55
@openclouders openclouders mentioned this pull request Sep 9, 2025
1 task
schweigisito pushed a commit that referenced this pull request Sep 10, 2025
* change: remove all opencloud_full references

* fix: remove leftovers of the opencloud_full stack
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants